Skip to content

Conversation

ernest-nowacki
Copy link
Contributor

@ernest-nowacki ernest-nowacki commented Oct 3, 2025

Adds CRE CLI simulation of hello-world workflow as part of the required health checks.

ernest-nowacki and others added 3 commits October 3, 2025 13:43
…cution of untrusted code

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@ernest-nowacki ernest-nowacki requested a review from Copilot October 3, 2025 12:25
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a smoke test workflow to validate SDK examples and updates documentation to remove broadcast flags from workflow simulation commands.

  • Adds comprehensive GitHub Actions smoke test workflow for continuous integration
  • Removes --broadcast flags from workflow simulation examples in documentation
  • Aligns documentation examples with actual test execution commands

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/smoke-test.yml Adds new CI workflow with system setup, dependency installation, and workflow simulation tests
packages/cre-sdk-examples/README.md Updates documentation to remove --broadcast flags from simulation commands

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ernest-nowacki ernest-nowacki marked this pull request as ready for review October 8, 2025 13:55
@ernest-nowacki ernest-nowacki requested a review from a team as a code owner October 8, 2025 13:55
Copy link
Contributor

@jeffrifwaldsmartcontract jeffrifwaldsmartcontract left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

It might be worth breaking out the different tests into separate jobs at some point. That is, it will eventually get annoying if you have to re-run the standard tests or workflow simulation when there is just a lint issue.

@ernest-nowacki ernest-nowacki merged commit c94f3d4 into main Oct 8, 2025
9 checks passed
@ernest-nowacki ernest-nowacki deleted the feat/smoke-test branch October 8, 2025 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants